components/airweave/package.json (1)

16-16: Update to the latest Airweave SDK version (0.6.46).

The dependency is currently pinned at @airweave/sdk@^0.1.50—upgrade to ^0.6.46 to gain recent bug fixes, improved runtime support, enhanced TypeScript types, retry behavior, and new helpers.

-    "@airweave/sdk": "^0.1.50",
+    "@airweave/sdk": "^0.6.46",

After upgrading, verify that all imports of AirweaveSDKClient and its method signatures still align with the new SDK.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

components/airweave/actions/create-collection/create-collection.mjs (1)

14-32: Props are well-defined. Consider adding format validation for readableId.

The props follow best practices with clear descriptions. The description prop is appropriately optional. The readableId description explains the expected format (lowercase, hyphens), but there's no programmatic enforcement.

Optionally, you could add pattern validation to catch format issues early:

 readableId: {
   type: "string",
   label: "Readable ID",
   description: "URL-friendly identifier for the collection (lowercase, hyphens allowed, e.g., 'customer-support-data'). This cannot be changed after creation.",
+  pattern: "^[a-z0-9-]+$",
 },

This would provide immediate feedback if the format is invalid, rather than letting the API reject it.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

components/airweave/airweave.app.mjs (2)

12-26: Stop paginating when no more collections are returned.

If collections.length === 0, skip doesn’t advance and the UI keeps requesting the same empty page. Omit context when no options remain.

-        return {
-          options: collections.map((collection) => ({
-            label: collection.name || collection.readable_id,
-            value: collection.readable_id,
-          })),
-          context: {
-            skip: params.skip + collections.length,
-          },
-        };
+        const options = collections.map((collection) => ({
+          label: collection.name || collection.readable_id,
+          value: collection.readable_id,
+        }));
+        return {
+          options,
+          context: options.length
+            ? { skip: params.skip + options.length }
+            : undefined,
+        };

---

33-52: Terminate pagination for source connections when no rows remain.

Same issue as above: return no context when the page is empty to stop infinite requests.

-        return {
-          options: connections.map((conn) => ({
-            label: conn.name || conn.id,
-            value: conn.id,
-          })),
-          context: {
-            skip: params.skip + connections.length,
-          },
-        };
+        const options = connections.map((conn) => ({
+          label: conn.name || conn.id,
+          value: conn.id,
+        }));
+        return {
+          options,
+          context: options.length
+            ? { skip: params.skip + options.length }
+            : undefined,
+        };

components/airweave/airweave.app.mjs (3)

2-2: Avoid JSON import assertions to satisfy Biome and maximize runtime compatibility.

Static analysis flags a parse error on the JSON import. Replace it with createRequire and pass a guarded frameworkVersion.

-import packageJson from "./package.json" assert { type: "json" };
+import { createRequire } from "module";
+const require = createRequire(import.meta.url);
+let frameworkVersion = "unknown";
+try {
+  frameworkVersion = require("./package.json")?.version || frameworkVersion;
+} catch {
+  // package.json may be pruned at runtime; fall back gracefully
+}

And update the client construction below:

-        frameworkVersion: packageJson.version,
+        frameworkVersion,

---

94-101: Minor: memoize the SDK client to avoid rebuilding per call (optional).

Tiny perf win and less object churn; rebuild only if auth/baseUrl changes.

-    _client() {
-      return new AirweaveSDKClient({
+    _client() {
+      if (this._cachedClient) return this._cachedClient;
+      this._cachedClient = new AirweaveSDKClient({
         apiKey: this._apiKey(),
         baseUrl: this._baseUrl(),
         frameworkName: "pipedream",
-        frameworkVersion: packageJson.version,
+        frameworkVersion,
       });
+      return this._cachedClient;
     },

---

103-176: Style nit: drop redundant await in return await client.* methods.

return await adds no value here and slightly harms stack traces. Consider returning the promise directly across these methods.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

components/airweave/airweave.app.mjs (2)

17-25: End pagination when no more collections are returned.

Empty pages currently repeat forever because context.skip doesn’t advance when collections.length === 0. Return no context when there are no options.

-        return {
-          options: collections.map((collection) => ({
-            label: collection.name || collection.readable_id,
-            value: collection.readable_id,
-          })),
-          context: {
-            skip: params.skip + collections.length,
-          },
-        };
+        const options = collections.map((collection) => ({
+          label: collection.name || collection.readable_id,
+          value: collection.readable_id,
+        }));
+        return {
+          options,
+          context: options.length
+            ? { skip: params.skip + options.length }
+            : undefined,
+        };

---

43-51: Terminate pagination for source connections on empty page.

Same issue: empty results loop indefinitely. Omit context when there are no options.

-        return {
-          options: connections.map((conn) => ({
-            label: conn.name || conn.id,
-            value: conn.id,
-          })),
-          context: {
-            skip: params.skip + connections.length,
-          },
-        };
+        const options = connections.map((conn) => ({
+          label: conn.name || conn.id,
+          value: conn.id,
+        }));
+        return {
+          options,
+          context: options.length
+            ? { skip: params.skip + options.length }
+            : undefined,
+        };

components/airweave/airweave.app.mjs (3)

86-176: Add concise JSDoc for public methods.

Document params and returns for app methods to align with Pipedream component guidelines.

As per coding guidelines.

---

93-100: Optional: avoid version drift.

Define a single const FRAMEWORK_VERSION = "0.1.0" (or read from package.json) and reuse it in _client().

---

11-53: Optional: DRY pagination return logic.

Extract a tiny helper to build { options, context } to reuse across option builders.

+  methods: {
+    _optionsPage(options, skip, pageSize) {
+      return {
+        options,
+        context: options.length ? { skip: skip + options.length } : undefined,
+      };
+    },
-      async options({ prevContext }) {
+      async options({ prevContext }) {
         const params = {
           skip: prevContext?.skip || 0,
           limit: 50,
         };
         const collections = await this.listCollections(params);
-        return {
-          options: collections.map((collection) => ({
-            label: collection.name || collection.readable_id,
-            value: collection.readable_id,
-          })),
-          context: {
-            skip: params.skip + collections.length,
-          },
-        };
+        const options = collections.map((c) => ({
+          label: c.name || c.readable_id,
+          value: c.readable_id,
+        }));
+        return this._optionsPage(options, params.skip, params.limit);
       },

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro